Skip to content

Import muCF cycle rate data and construct muCF model #2240

Open
stognini wants to merge 38 commits intoceleritas-project:developfrom
stognini:mucf-material-data
Open

Import muCF cycle rate data and construct muCF model #2240
stognini wants to merge 38 commits intoceleritas-project:developfrom
stognini:mucf-material-data

Conversation

@stognini
Copy link
Member

@stognini stognini commented Feb 6, 2026

This PR completes another component of #1508 . It adds the cycle rate data for dd, dt, and tt fusion interactions in inp::, and correctly constructs the host/device data used by the muCF model.

- Move types to new mucf/Types.hh
- Move MucfParticles::from_params to model.cc as a free function
- Move all static data to MucfPhysics.cc
- Refactor/Remove types in DTMixMucfData
- Remove CELER_EXPECT(mu_minus) during MucfProcess construction
- Fix import data
- Refactor the material calculator to behave as a material inserter
- Interactors: define number of secondaries as EnumArrays
- Rename internal material id to MucfMatId
- Remove find MucfMatId function from DTMixMucfData and make it a lambda
  in the executor
- Fix integral_xs bool
- Fwd declare material and particle params
- Remove trailing includes
@stognini stognini added enhancement New feature or request physics Particles, processes, and stepping algorithms labels Feb 6, 2026
@stognini stognini mentioned this pull request Feb 6, 2026
23 tasks
@github-actions
Copy link

github-actions bot commented Feb 6, 2026

Test summary

 5 928 files   9 466 suites   18m 56s ⏱️
 2 153 tests  2 125 ✅  28 💤 0 ❌
32 338 runs  32 180 ✅ 158 💤 0 ❌

Results for commit 52f8154.

♻️ This comment has been updated with latest results.

@stognini stognini removed the request for review from drbenmorgan February 6, 2026 20:47
@codecov
Copy link

codecov bot commented Feb 6, 2026

Codecov Report

❌ Patch coverage is 92.85714% with 23 lines in your changes missing coverage. Please review.
✅ Project coverage is 87.17%. Comparing base (599cdbc) to head (52f8154).

Files with missing lines Patch % Lines
...ucf/model/detail/EquilibrateDensitiesCalculator.cc 81.55% 15 Missing and 4 partials ⚠️
...eleritas/mucf/model/detail/MucfMaterialInserter.cc 95.50% 1 Missing and 3 partials ⚠️
Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #2240      +/-   ##
===========================================
+ Coverage    86.87%   87.17%   +0.29%     
===========================================
  Files         1331     1334       +3     
  Lines        42102    42366     +264     
  Branches     12951    13014      +63     
===========================================
+ Hits         36578    36932     +354     
+ Misses        4319     4238      -81     
+ Partials      1205     1196       -9     
Files with missing lines Coverage Δ
src/celeritas/inp/MucfPhysics.cc 100.00% <100.00%> (ø)
src/celeritas/inp/MucfPhysics.hh 0.00% <ø> (-10.00%) ⬇️
src/celeritas/mucf/data/DTMixMucfData.hh 100.00% <100.00%> (+8.33%) ⬆️
src/celeritas/mucf/interactor/DTMucfInteractor.hh 93.33% <ø> (ø)
src/celeritas/mucf/model/DTMixMucfModel.cc 66.12% <100.00%> (+66.12%) ⬆️
...ucf/model/detail/EquilibrateDensitiesCalculator.hh 100.00% <100.00%> (ø)
.../celeritas/mucf/model/detail/InterpolatorHelper.cc 100.00% <100.00%> (ø)
...eleritas/mucf/model/detail/MucfMaterialInserter.cc 94.64% <95.50%> (+94.64%) ⬆️
...ucf/model/detail/EquilibrateDensitiesCalculator.cc 81.55% <81.55%> (ø)

... and 10 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request physics Particles, processes, and stepping algorithms

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant